[njngwn] WEEK 07 solutions by njngwn · Pull Request #1885 · DaleStudy/leetcode-study

njngwn · 2025-09-03T13:37:13Z

답안 제출 문제

작성자 체크 리스트

Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

검토자 체크 리스트

Important

본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!

바로 이전에 올라온 PR에 본인을 코드 리뷰어로 추가해주세요.
본인이 검토해야하는 PR의 답안 코드에 피드백을 주세요.
토요일 전까지 PR을 병합할 수 있도록 승인해주세요.

This reverts commit 75361b7.

hi-rachel

한 문제는 아직 안올리신 건가요?
코드가 깔끔해서 java인데 어느 정도 이해했습니다. 👍🏻

njngwn · 2025-09-04T11:35:51Z

한 문제는 아직 안올리신 건가요? 코드가 깔끔해서 java인데 어느 정도 이해했습니다. 👍🏻

아뇨. 다른 문제에 비해서 좀더 빨리 푸시해놔서 그런건지 왜그런건지ㅠㅠ PR에 안올라가서 revert까지 했는데도, 파일로 반영이 안되네요ㅠ (solution: reverse linked list) 이 커밋의 파일인데.. 이유를 모르겠네요..

hi-rachel · 2025-09-05T13:59:09Z

한 문제는 아직 안올리신 건가요? 코드가 깔끔해서 java인데 어느 정도 이해했습니다. 👍🏻

아뇨. 다른 문제에 비해서 좀더 빨리 푸시해놔서 그런건지 왜그런건지ㅠㅠ PR에 안올라가서 revert까지 했는데도, 파일로 반영이 안되네요ㅠ (solution: reverse linked list) 이 커밋의 파일인데.. 이유를 모르겠네요..

@njngwn revert를 해서 파일이 삭제된거 같은데요??
기록에 남기만 하면 상관없을거 같은데 해당 문제가 풀었다고 기록이 안남는다면 파일 다시 한 번 커밋 해보시면 좋을 것 같습니다!

hi-rachel · 2025-09-05T14:03:42Z

한 문제는 아직 안올리신 건가요? 코드가 깔끔해서 java인데 어느 정도 이해했습니다. 👍🏻

아뇨. 다른 문제에 비해서 좀더 빨리 푸시해놔서 그런건지 왜그런건지ㅠㅠ PR에 안올라가서 revert까지 했는데도, 파일로 반영이 안되네요ㅠ (solution: reverse linked list) 이 커밋의 파일인데.. 이유를 모르겠네요..

@njngwn revert를 해서 파일이 삭제된거 같은데요?? 기록에 남기만 하면 상관없을거 같은데 해당 문제가 풀었다고 기록이 안남는다면 파일 다시 한 번 커밋 해보시면 좋을 것 같습니다!

아 그 이후에 다시 커밋도 하셨었군요.. 뭔가 revert하면서 꼬인거 같은데.. 혹시 기록이 안남는다고 하시면 메인테이너분에게 문의하거나 다시 깔끔히 (로컬에서 revert 전으로 깔끔히 날려버리고 + 코드 백업은 하고) PR 올려서 저 지정해주시면 바로 승인해드릴게요!

Lustellz · 2025-09-06T13:27:01Z

헤헤... 제가 한 문제 겨우 푼(?) 수준이라 코드 리뷰를 해도 되는지 잘 모르겠네요... 고생 많으셨습니다!

Lustellz · 2025-09-06T13:32:49Z

longest-substring-without-repeating-characters/njngwn.java

+            // if the letter is not in the hashset, then add it into hashset and move end pointer
+            if (!letterSet.contains(s.charAt(end))) {
+                letterSet.add(s.charAt(end));
+                ++end;


제가 문제를 안 읽어서 잘은 모르겠는데 보통 후위 표기를 쓰는 걸 봤어서 전위 표기를 하신 이유가 궁금합니다~

아, 그냥 제 코딩습관인데 ++i가 i++보다 조금 더 빠르다는 말을 들어서요..!! 근데 자바라서 상관없을 것 같기도 하고, 요즘 컴파일러는 상관없다구 하네요. https://ssocoit.tistory.com/51

Lustellz · 2025-09-06T13:34:16Z

longest-substring-without-repeating-characters/njngwn.java

문제를 안 읽었지만 요구하는 게 뭔지 알 거 같네요! 주석의 설명이 짧지만 명확하고 코드 역시 깔끔해서 가독성이 좋다고 생각합니다.

감사합니다!!!

Lustellz · 2025-09-06T13:38:27Z

number-of-islands/njngwn.java

앗! DFS로 푸는 문제들이 나왔군요. 좀 더 시간을 들여서 저도 풀었다면 좋았을텐데... 쪼끔 아쉬운 마음이 남네요.
고생하셨습니다!

꼼꼼하게 봐주셔서 감사해요!! 다음 기수에 한 번 더 도전하셔서 풀어보셔도 좋을 것 같아요!!

Lustellz · 2025-09-06T13:41:03Z

set-matrix-zeroes/njngwn.java

+        ArrayList<Integer> rowList = new ArrayList<>();
+        ArrayList<Integer> colList = new ArrayList<>();
+
+        for (int m = 0; m < matrix.length; ++m) {


여기서도 전위 표기를 하셨네요! 선택한 이유가 있으신지 궁금합니다~

njngwn · 2025-09-07T12:00:21Z

한 문제는 아직 안올리신 건가요? 코드가 깔끔해서 java인데 어느 정도 이해했습니다. 👍🏻

아뇨. 다른 문제에 비해서 좀더 빨리 푸시해놔서 그런건지 왜그런건지ㅠㅠ PR에 안올라가서 revert까지 했는데도, 파일로 반영이 안되네요ㅠ (solution: reverse linked list) 이 커밋의 파일인데.. 이유를 모르겠네요..

@njngwn revert를 해서 파일이 삭제된거 같은데요?? 기록에 남기만 하면 상관없을거 같은데 해당 문제가 풀었다고 기록이 안남는다면 파일 다시 한 번 커밋 해보시면 좋을 것 같습니다!

아 그 이후에 다시 커밋도 하셨었군요.. 뭔가 revert하면서 꼬인거 같은데.. 혹시 기록이 안남는다고 하시면 메인테이너분에게 문의하거나 다시 깔끔히 (로컬에서 revert 전으로 깔끔히 날려버리고 + 코드 백업은 하고) PR 올려서 저 지정해주시면 바로 승인해드릴게요!

solution: reverse linked list 이 커밋으로 올라가 있어요..!! 감사합니다!

Jeongwon Na added 4 commits September 3, 2025 15:11

solved week 07

51d3c70

add last line

7f88af9

Revert "solution: reverse-linked-list using iteration"

f5ab6f7

This reverts commit 75361b7.

solution: reverse linked list

13cd15b

github-project-automation bot added this to 리트코드 스터디 5기 Sep 3, 2025

github-project-automation bot moved this to Solving in 리트코드 스터디 5기 Sep 3, 2025

github-actions bot added the java label Sep 3, 2025

github-actions bot assigned njngwn Sep 3, 2025

njngwn moved this from Solving to In Review in 리트코드 스터디 5기 Sep 3, 2025

hi-rachel self-requested a review September 4, 2025 05:24

hi-rachel approved these changes Sep 4, 2025

View reviewed changes

Lustellz approved these changes Sep 6, 2025

View reviewed changes

njngwn merged commit d62d075 into DaleStudy:main Sep 7, 2025
3 checks passed

github-project-automation bot moved this from In Review to Completed in 리트코드 스터디 5기 Sep 7, 2025

Uh oh!

Conversation

njngwn commented Sep 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

답안 제출 문제

작성자 체크 리스트

검토자 체크 리스트

Uh oh!

hi-rachel left a comment

Choose a reason for hiding this comment

Uh oh!

njngwn commented Sep 4, 2025

Uh oh!

hi-rachel commented Sep 5, 2025

Uh oh!

hi-rachel commented Sep 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Lustellz commented Sep 6, 2025

Uh oh!

Lustellz Sep 6, 2025

Choose a reason for hiding this comment

Uh oh!

njngwn Sep 7, 2025

Choose a reason for hiding this comment

Uh oh!

Lustellz Sep 6, 2025

Choose a reason for hiding this comment

Uh oh!

njngwn Sep 7, 2025

Choose a reason for hiding this comment

Uh oh!

Lustellz Sep 6, 2025

Choose a reason for hiding this comment

Uh oh!

njngwn Sep 7, 2025

Choose a reason for hiding this comment

Uh oh!

Lustellz Sep 6, 2025

Choose a reason for hiding this comment

Uh oh!

njngwn commented Sep 7, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

njngwn commented Sep 3, 2025 •

edited

Loading

hi-rachel commented Sep 5, 2025 •

edited

Loading